Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add PredX project tvl on ZKLlink #11192

Merged
merged 8 commits into from
Aug 5, 2024
Merged

Conversation

Junpeng-Wang
Copy link
Contributor

Name (to be shown on DefiLlama): PredX.ai
Twitter Link: https://x.com/predx_ai
Website Link: https://predx.ai/
Logo (High resolution, will be shown with rounded borders): https://x.com/PredX_AI/photo
Current TVL: 1.9969290720000001
Chain: ZKLink
Short Description (to be shown on DefiLlama): Multichain prediction market, powered by AI.
Category: Prediction Market

@llamatester
Copy link

The adapter at projects/predx exports TVL:

zklink                    2

total                    1.9966713299999999 

@realdealshaman
Copy link
Contributor

Hey @Junpeng-Wang , why didn't you add the other chains like base, sei, and the others to the adapter?

@Junpeng-Wang
Copy link
Contributor Author

Hey @Junpeng-Wang , why didn't you add the other chains like base, sei, and the others to the adapter?

Sure. I will add more and create another PR. Before that, (1) shall I worry about the current failing check? (2) does the sumTokensExport function also support sei? It gives me error like "PromisePoolError: Failed to get https://sei-rest.publicnode.com/cosmos/bank/v1beta1/balances/...?pagination.limit=1000"

Thanks

@llamatester
Copy link

The adapter at projects/predx exports TVL:

zklink                    2

total                    1.992033972 

@g1nt0ki
Copy link
Member

g1nt0ki commented Aug 5, 2024

hi @Junpeng-Wang, it does support sei, think issue was, it was picking cosmos helper for sei instead of evm helper, fixing it.

@llamatester
Copy link

The adapter at projects/predx exports TVL:

zklink                    2

total                    2.0159819999999997 

@g1nt0ki g1nt0ki self-assigned this Aug 5, 2024
@llamatester
Copy link

The adapter at projects/predx exports TVL:

base                      3.05 k
btr                       1.11 k
bsc                       12
linea                     10
mantle                    2
zklink                    2

total                    4.19 k 

@llamatester
Copy link

The adapter at projects/predx exports TVL:

base                      3.05 k
btr                       1.11 k
bsc                       12
linea                     10
mantle                    2
zklink                    2

total                    4.19 k 

@llamatester
Copy link

The adapter at projects/predx exports TVL:

base                      3.05 k
btr                       1.11 k
bsc                       12
linea                     10
zklink                    2
mantle                    2

total                    4.19 k 

@Junpeng-Wang
Copy link
Contributor Author

hi @Junpeng-Wang, it does support sei, think issue was, it was picking cosmos helper for sei instead of evm helper, fixing it.

Thanks. I have updated the code to add all other chains we have. The sei chain has been commented out. Please let me know if any other changes are required.

@g1nt0ki g1nt0ki merged commit 4e82aa4 into DefiLlama:main Aug 5, 2024
1 check passed
@g1nt0ki
Copy link
Member

g1nt0ki commented Aug 5, 2024

thanks @Junpeng-Wang

@llamatester
Copy link

The adapter at projects/predx exports TVL:

base                      3.05 k
btr                       1.10 k
sei                       27
bsc                       12
linea                     10
zklink                    2
mantle                    2

total                    4.21 k 

tlatkdgus1 pushed a commit to bifrost-platform/DefiLlama-Adapters that referenced this pull request Nov 25, 2024
* add PredX project tvl on zklink

* test fix

* minor fix

* add more chains of PredX

* minor style change

* remove duplicated btr

* Update index.js

---------

Co-authored-by: g1nt0ki <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants